-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[WIP] #3412 documenting overriden options #3417
Conversation
nykopol
commented
Jan 3, 2014
Q | A |
---|---|
Doc fix? | no |
New docs? | yes |
Applies to | 2.3 |
Fixed tickets | #3412 |
@weaverryan yes it's my mistake i did not start in an other branch. When you say a 'rebase' you think a new PR ? |
Hey @nykopol! No problem :). You can actually rebase your git checkout overridden_options
git fetch upstream
git rebase upstream/master
git push origin overridden_options --force As soon as you push to the Cheers! |
…y-docs into overriden_options Conflicts: reference/forms/types/options/checkbox_compound.rst.inc reference/forms/types/options/checkbox_empty_data.rst.inc
Thanks @weaverryan for you help. I don't had conflict on rebase but on push. I hope it's ok now. |
Sorry @nykopol it's still conflicting. |
Hey @nykopol! Yea, it's still conflicting unfortunately :). Did you fetch from the "symfony/symfony-docs" repository and rebase against its master? Let me know! Thanks! |
Hi guys! I've rebased and re-opened this PR at #3506 for review. I'll close this :). Thanks! |
This PR was merged into the 2.3 branch. Discussion ---------- Nykopol overriden options See #3417 - this is just a rebase to continue it :) | Q | A | ------------- | --- | Doc fix? | no | New docs? | yes | Applies to | 2.3 | Fixed tickets | #3412 Thanks! Commits ------- e574e97 Removing an extra line thanks to @xabbuh 4f6cb9d Some additional tweaks and probably some merge conflict fixes 362c8b4 document overriden options of choice field 70feefd document overriden options of checkbox c62cd13 correct overriden option of button and add checkbox overriden option ab4aafe correct overriden option of button and add checkbox overriden option